Use correct leading slash in common.ps1 repo root resolution#10694
Use correct leading slash in common.ps1 repo root resolution#10694
Conversation
benbp
added
the
Central-EngSys
benbp
added
the
Central-EngSys
There was a problem hiding this comment.
Pull Request Overview
The PR fixes the repository root resolution in common.ps1 by correcting the number of parent traversals and standardizing the path string.
- Adjusts the relative path from four levels up to three to correctly target the repo root
- Switches from backslash-based
..\..\..\..notation to forward-slash../../../notation
|
The following pipelines have been queued for testing: |
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#10694 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
3 participants
Use correct leading slash in common.ps1 repo root resolution. You need a leading slash after $PSScriptRoot, hence why the current code has to use 4
..instead of 3 (the first ends up getting ignored).